XML::RSS

Shlomi Fish on 2006-10-18T10:27:55

I noticed that at the moment, XML-RSS has over 30 open bugs. So trying to help with this important module, I started to write patches for it.

At first I imported the files into the web-cpan repository , and patched it there. Then I wrote a few patches by working on the working copy of a checkout from the svn.perl.org repository, but that turned out not to scale too well. Eventually I decided to use svk instead, and wrote two patches using it.

I kept some notes in a text file about my progress. Here they are:

Bugs covered:

• Bug #21740
  
  • has a test
  • has a patch.
  • not in repos
• Bug #19420
  
  • has a test.
  • has a patch.
  • in repos.
• http://rt.cpan.org/Public/Bug/Display.html?id=21048
  
  • same as http://rt.cpan.org/Ticket/Display.html?id=19420
• Bug #21046
  
  • works for me.
    • should test on Windows.
  • works for the author.
    • TODO: should be closed.
• Bug #7325
  
  • works for me.
    • cannot confirm.
  • very old bug.
  • suggest to close.
• Bug #19640
  
  • invalid
    • should be closed.
• Bug #19417
  
  • works with a simple script.
  • problem is there is a different date format.
  • Fixed as a patch.
    • TODO : Apply.
• Bug #18755
  
  • Patch sent
  • TODO : Apply.
• Bug #18907
  
  • I don't understand it too well - skipping.
• Bug #5024
  
  • Wrote a patch for it:
  • TODO : Apply.

Ask Bjørn Hansen (the XML::RSS maintainer) has already applied a patch, but there are more in the Request Tracker. And there are still a lot of bugs to cover.

---

Don't close bugs you haven't tested

speters on 2006-10-18T13:56:29

Bug #21046 is a real bug. Unfortunately, it is a duplicate of #7343, which I opened over two years ago. It is also still happening on Windows today.

Re:Don't close bugs you haven't tested

speters on 2006-10-18T14:06:00

I've actually added a patch for this problem here.

Re:Don't close bugs you haven't tested

ask on 2006-10-19T20:17:45

Thanks,

committed.